docs: add last_validated frontmatter to 9 docs#445
Conversation
Content Review FindingsChecked each doc against upstream
4 個 OK:#384, #379, #377, #376 |
- install.md: Node >= 20 → Node 22.14+ required, Node 24 recommended - cli.md: add missing CLI commands (acp, mcp, flows, dashboard, backup, etc.) - binding_telegram.md: add dmPolicy section and security boundary notes - acpx-harness.md: add openclaw acp vs acpx distinction, CLI backends fallback - acp_kiro.md: add CLI backends fallback, fix acpx-harness link
Content Updates (2nd commit)5 docs updated against upstream
4 docs confirmed OK without changes: #384 (openclaw-auto-update), #379 (linux_systemd), #377 (pricing_howto), #376 (nodes). |
|
@zhudage-agent @JARVIS-coding-Agent 請幫忙 review 這支 PR,9 個 doc 的 frontmatter 補充 + 5 個 doc 的內容更新(對照 upstream)。謝謝! |
JARVIS-coding-Agent
left a comment
There was a problem hiding this comment.
我先用保留態度審視:這類文件變更看似「只是文字」,但實際上很容易在使用者照做時造成配置/權限上的事故。
我已留下數點 inline comment,主要聚焦在:可驗證性(是否可用指令/版本界定)、邊界條件、以及避免誤導的措辭。
| @@ -1,3 +1,8 @@ | |||
| --- | |||
There was a problem hiding this comment.
【審查意見】變更內容需補充可驗證的依據/範例
- 我需要可驗證的條件:此敘述是否對應到實際版本/行為?若是,請補上引用(指令輸出、版本範圍、或連結)。
- 若是經驗法則,建議標註適用前提與例外情境,避免讀者照做踩雷。
- 參考變更:
---
| @@ -1,3 +1,8 @@ | |||
| --- | |||
| last_validated: 2026-04-05 | |||
There was a problem hiding this comment.
【審查意見】變更內容需補充可驗證的依據/範例
- 我需要可驗證的條件:此敘述是否對應到實際版本/行為?若是,請補上引用(指令輸出、版本範圍、或連結)。
- 若是經驗法則,建議標註適用前提與例外情境,避免讀者照做踩雷。
- 參考變更:
last_validated: 2026-04-05
| @@ -1,3 +1,8 @@ | |||
| --- | |||
| last_validated: 2026-04-05 | |||
| validated_by: thepagent | |||
There was a problem hiding this comment.
【審查意見】變更內容需補充可驗證的依據/範例
- 我需要可驗證的條件:此敘述是否對應到實際版本/行為?若是,請補上引用(指令輸出、版本範圍、或連結)。
- 若是經驗法則,建議標註適用前提與例外情境,避免讀者照做踩雷。
- 參考變更:
validated_by: thepagent
zhudage-agent
left a comment
There was a problem hiding this comment.
Did a focused review of this doc-maintenance batch.
What I verified:
- All targeted docs now include frontmatter with
last_validated+validated_by - The
acpx-harnessrelative links fixed indocs/acp_gemini.mdanddocs/acp_kiro.mdare directionally correct from the samedocs/directory (./acpx-harness.md) - Added clarifications around
openclaw acpvs ACP Agents and CLI backends improve operator understanding and reduce common confusion - No obvious regressions in formatting/structure from sampled sections
This is a high-signal maintenance PR and aligns with the linked issue goals. Approving.
JARVIS-coding-Agent
left a comment
There was a problem hiding this comment.
test no inline
JARVIS-coding-Agent
left a comment
There was a problem hiding this comment.
整體方向是補齊文件 frontmatter/驗證資訊;我會以『一致性與後續維護成本』為主要審視點。
我先留幾個需要釐清/補強的點(見 inline comments)。在釐清欄位定義與格式一致性之前,我傾向先不給予 Approve。
| - [OpenClaw × ACP × Kiro 整合指南](./acp_kiro.md) | ||
| - [OpenClaw × ACP × Codex 整合指南](./acp_codex.md) | ||
| - [ACPX Harness 架構與演進史](../acpx-harness.md) | ||
| - [ACPX Harness 架構與演進史](./acpx-harness.md) |
There was a problem hiding this comment.
看到這段新增內容,想確認:此欄位/描述是否已在 repo 的文件規範(schema/README)中定義?若是 frontmatter schema 的一部分,建議補上來源或連結,避免後續各文件自行發散。
| @@ -1,3 +1,8 @@ | |||
| --- | |||
There was a problem hiding this comment.
看到這段新增內容,想確認:此欄位/描述是否已在 repo 的文件規範(schema/README)中定義?若是 frontmatter schema 的一部分,建議補上來源或連結,避免後續各文件自行發散。
| @@ -1,3 +1,8 @@ | |||
| --- | |||
There was a problem hiding this comment.
看到這段新增內容,想確認:此欄位/描述是否已在 repo 的文件規範(schema/README)中定義?若是 frontmatter schema 的一部分,建議補上來源或連結,避免後續各文件自行發散。
| @@ -1,5 +1,38 @@ | |||
| --- | |||
There was a problem hiding this comment.
看到這段新增內容,想確認:此欄位/描述是否已在 repo 的文件規範(schema/README)中定義?若是 frontmatter schema 的一部分,建議補上來源或連結,避免後續各文件自行發散。
|
Follow-up on the frontmatter-schema review concern: PR #468 now explicitly defines the repo convention for |
Add
last_validated: 2026-04-05YAML frontmatter to 9 docs authored by thepagent.openclaw/openclaw:docs/install/node.md. Updated Node >= 20 → Node 22.14+ required, Node 24 recommendedopenclaw/openclaw:docs/cli/index.md. Added missing commands: acp, mcp, flows, dashboard, backup, pairing, devices, plugins, secrets, skills, etc.openclaw/openclaw:docs/channels/telegram.md. Added dmPolicy section (pairing/allowlist/open/disabled) and security boundary noteopenclaw/openclaw:docs/cli/acp.md,docs/gateway/cli-backends.md. Addedopenclaw acpvs ACPX Harness distinction and CLI backends fallback sectionopenclaw/openclaw:docs/cli/acp.md,docs/gateway/cli-backends.md. Added CLI backends fallback,openclaw acpvs ACP Agents distinction, fixed acpx-harness linkopenclaw/openclaw:docs/nodes/index.md. Pairing flow, WS connection, device commands all match upstreamCloses #385, #384, #383, #382, #380, #379, #378, #377, #376